Model generator USAGE doc improvement #10092
Conversation
|
|
||
| `rails generate model user pseudo:string{30}` | ||
| `rails generate model user pseudo:string:uniq` |
There was a problem hiding this comment.
It'd be good to have a :index option here too.
|
Looks good, just minor comments. Thanks. |
|
Yeah, sorry, I read that as "Then you can" 10 times. I have those fixes ready, should I just push a second commit? Or you guys want them squashed into a single commit? I don't remember the preference. |
|
I went ahead and pushed a new commit, let me know if you want it squashed instead. |
|
Usually they should be squashed because they related to the same change. If On Thu, Apr 4, 2013 at 4:17 PM, Jason King notifications@github.com wrote:
At. |
|
I do the same thing all the time: I recently submitted some docs to Rust that had THREE ROUNDS of typos. |
* Added information about syntax for precision/scale of decimals * Removed incorrect information about being able to set `default` * Added more examples of usage
|
Done, thanks. |
Model generator USAGE doc improvement
This started with me noticing that the
defaultoption mentioned in the help docs forrails g modeldidn't actually work. I removed that, and improved the docs for the other options:default